Skip to content

[feat][backend]: add not_query_annotation option to list spans request#503

Merged
kidkidkid merged 4 commits intomainfrom
feat/trace_refactor
Apr 22, 2026
Merged

[feat][backend]: add not_query_annotation option to list spans request#503
kidkidkid merged 4 commits intomainfrom
feat/trace_refactor

Conversation

@kidkidkid
Copy link
Copy Markdown
Collaborator

@kidkidkid kidkidkid commented Apr 21, 2026

What type of PR is this?

Check the PR title

  • This PR title match the format: [<type>][<scope>] <description>. For example: [fix][backend] flaky fix
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Add documentation if the current PR requires user awareness at the usage level.
  • This PR is written in English. PRs not in English will not be reviewed.

(Optional) Translate the PR title into Chinese

(Optional) More detailed description for this PR(en: English/zh: Chinese)

en:
zh(optional):

(Optional) Which issue(s) this PR fixes

@kidkidkid kidkidkid changed the title [feat[[backend]: add not_query_annotation option to list spans request [feat][backend]: add not_query_annotation option to list spans request Apr 21, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 21, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 84.21053% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ckend/modules/observability/application/openapi.go 0.00% 5 Missing and 1 partial ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #503      +/-   ##
==========================================
- Coverage   77.07%   76.02%   -1.05%     
==========================================
  Files         649      649              
  Lines       72039    86949   +14910     
==========================================
+ Hits        55521    66106   +10585     
- Misses      13199    17522    +4323     
- Partials     3319     3321       +2
Flag Coverage Δ
unittests 76.02% <84.21%> (-1.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...bservability/domain/trace/service/trace_service.go 86.86% <100.00%> (-1.85%) ⬇️
backend/modules/observability/infra/repo/trace.go 82.82% <100.00%> (-3.09%) ⬇️
...ckend/modules/observability/application/openapi.go 81.76% <0.00%> (-2.46%) ⬇️

... and 583 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10d5a75...1a35ba4. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

taoyifan89
taoyifan89 previously approved these changes Apr 22, 2026
View reviewed changes
@kidkidkid kidkidkid merged commit 45bc9a9 into main Apr 22, 2026
16 checks passed
@kidkidkid kidkidkid deleted the feat/trace_refactor branch April 22, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Ev4nFeng Ev4nFeng Ev4nFeng approved these changes

@taoyifan89 taoyifan89 taoyifan89 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kidkidkid @Ev4nFeng @taoyifan89